Skip to content

feat: add mattermost bot personal access token to wind-tunnel repo#76

Merged
mattyg merged 1 commit intomainfrom
feat/mattermost-bot-access-token
Mar 27, 2026
Merged

feat: add mattermost bot personal access token to wind-tunnel repo#76
mattyg merged 1 commit intomainfrom
feat/mattermost-bot-access-token

Conversation

@mattyg
Copy link
Copy Markdown
Member

@mattyg mattyg commented Mar 27, 2026 •
edited by coderabbitai bot
Loading

Summary by CodeRabbit

  • New Features
    • Added GitHub Actions secret support for Mattermost bot personal access token, enabling bot-based notifications for the wind-tunnel repository.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 27, 2026 •
edited
Loading

Caution

Review failed

Pull request was closed or merged during review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 7d77632f-4350-47ee-8b0f-b7d79468ae4a

📥 Commits

Reviewing files that changed from the base of the PR and between 508364f and 7c69c8f.

📒 Files selected for processing (2)
  • Pulumi.github.yaml
  • main.go

Walkthrough

This PR adds a new GitHub Actions secret provisioning capability by introducing a secure Pulumi configuration entry and a corresponding helper function that creates the HOLOCHAIN_NOTIFIER_MATTERMOST_BOT_PERSONAL_ACCESS_TOKEN secret for the wind-tunnel repository, following the established pattern of similar secret provisioners.

Changes

Cohort / File(s) Summary
Mattermost Bot Secret Provisioning
Pulumi.github.yaml, main.go		 Added new secure config parameter holochain:holochainNotifierMattermostBotPersonalAccessToken and corresponding helper function AddHolochainNotifierMattermostBotPersonalAccessToken() that provisions a GitHub Actions secret for the wind-tunnel repository with encryption-aware resource configuration.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

Suggested reviewers

  • jost-s
  • ThetaSinner
🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: adding a Mattermost bot personal access token configuration to the wind-tunnel repository.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/mattermost-bot-access-token

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@mattyg mattyg requested a review from a team March 27, 2026 01:24
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 27, 2026 •
edited
Loading

🍹 preview on holochain/holochain/github

Pulumi report

View in Pulumi Cloud

  Previewing update (holochain/github)

View Live: https://app.pulumi.com/holochain/holochain/github/previews/85e6b5f5-27d6-4ddc-9789-8108787a978d

pulumi:pulumi:Stack: (same)
  [urn=urn:pulumi:github::holochain::pulumi:pulumi:Stack::holochain-github]
  + github:index/actionsSecret:ActionsSecret: (create)
      [urn=urn:pulumi:github::holochain::github:index/actionsSecret:ActionsSecret::wind-tunnel-holochain-notifier-mattermost-bot-personal-access-token]
      [provider=urn:pulumi:github::holochain::pulumi:providers:github::default_6_11_0::dce38a79-6242-43ba-a5ff-26fae2b6f93c]
      destroyOnDrift: true
      plaintextValue: [secret]
      repository    : "wind-tunnel"
      secretName    : "HOLOCHAIN_NOTIFIER_MATTERMOST_BOT_PERSONAL_ACCESS_TOKEN"
Resources:
  + 1 to create
  377 unchanged

@mattyg mattyg force-pushed the feat/mattermost-bot-access-token branch from 2bf9101 to 7c69c8f Compare March 27, 2026 16:31
@cocogitto-bot
Copy link
Copy Markdown

cocogitto-bot bot commented Mar 27, 2026

✔️ 7c69c8f - Conventional commits check succeeded.

@mattyg mattyg merged commit ae253f0 into main Mar 27, 2026
3 of 4 checks passed
@mattyg mattyg deleted the feat/mattermost-bot-access-token branch March 27, 2026 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ThetaSinner ThetaSinner ThetaSinner approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mattyg @ThetaSinner